Skip to content

feat: Sidebar relooking#3792

Merged
doubleface merged 1 commit intomasterfrom
feat/newSidebar
Apr 8, 2026
Merged

feat: Sidebar relooking#3792
doubleface merged 1 commit intomasterfrom
feat/newSidebar

Conversation

@doubleface
Copy link
Copy Markdown
Contributor

@doubleface doubleface commented Apr 8, 2026

Capture d’écran du 2026-04-08 15-58-18

https://www.notion.so/linagora/Sidebar-design-changes-33062718bad1809697d4e0758874cb28

Summary by CodeRabbit

  • Localization

    • Updated navigation labels across locales for clearer wording (EN: "My Drive", "Recents", "Bin"; FR: "Mon Drive"; RU: "Мой диск"; VI: "Ổ đĩa của tôi").
  • Style

    • Replaced the drive/folder navigation icon for a refreshed look.
    • Slightly reduced filename text size in favorites for improved visual hierarchy.

codescene-delta-analysis[bot]

This comment was marked as outdated.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ad6fd1b0-9eb1-4fe4-824e-336b87ca4893

📥 Commits

Reviewing files that changed from the base of the PR and between 87b41b8 and f882061.

📒 Files selected for processing (7)
  • src/locales/en.json
  • src/locales/fr.json
  • src/locales/ru.json
  • src/locales/vi.json
  • src/modules/navigation/FavoriteListItem.tsx
  • src/modules/navigation/Nav.jsx
  • src/modules/views/useUpdateDocumentTitle.spec.js
✅ Files skipped from review due to trivial changes (6)
  • src/modules/navigation/Nav.jsx
  • src/modules/navigation/FavoriteListItem.tsx
  • src/locales/ru.json
  • src/locales/vi.json
  • src/locales/fr.json
  • src/locales/en.json

Walkthrough

Updated localization strings for the navigation labels across multiple locale files and small UI tweaks. In src/locales/en.json: Nav.item_drive "Files" → "My Drive", Nav.item_recent "Recent" → "Recents", Nav.item_trash "Trash" → "Bin". In src/locales/fr.json: Nav.item_drive "Fichiers" → "Mon Drive". In src/locales/ru.json: Nav.item_drive "Файлы" → "Мой диск". In src/locales/vi.json: Nav.item_drive "Tệp" → "Ổ đĩa của tôi". In src/modules/navigation/FavoriteListItem.tsx a Typography element gained className="u-fz-small". In src/modules/navigation/Nav.jsx some icon imports were replaced and the /folder nav item now renders a cloud icon. Tests updated to reflect "Trash" → "Bin" in document title expectations.

Suggested reviewers

  • rezk2ll
  • lethemanh
  • JF-Cozy
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Sidebar relooking' is vague and generic, lacking specificity about the actual changes made to the sidebar UI elements. Consider using a more descriptive title that specifies the changes, such as 'refactor: Update navigation labels and sidebar icons' or 'feat: Rebrand navigation items in sidebar'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/newSidebar

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/locales/vi.json`:
- Line 3: Replace the English label for Nav.item_drive in the Vietnamese locale
by updating the "item_drive" key in src/locales/vi.json from "My Drive" to a
proper Vietnamese string (e.g., "Ổ đĩa của tôi") so the sidebar label is fully
localized; locate the "item_drive" entry and change its value accordingly.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7f01315e-75a8-4d46-8a36-85346ea0c6f1

📥 Commits

Reviewing files that changed from the base of the PR and between c4c01dd and f788978.

📒 Files selected for processing (6)
  • src/locales/en.json
  • src/locales/fr.json
  • src/locales/ru.json
  • src/locales/vi.json
  • src/modules/navigation/FavoriteListItem.tsx
  • src/modules/navigation/Nav.jsx

Comment thread src/locales/vi.json Outdated
codescene-delta-analysis[bot]

This comment was marked as outdated.

@doubleface doubleface enabled auto-merge (rebase) April 8, 2026 14:32
Copy link
Copy Markdown

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gates Passed
3 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: The Bare Minimum
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@bundlemon
Copy link
Copy Markdown

bundlemon bot commented Apr 8, 2026

BundleMon

Unchanged files (21)
Status Path Size Limits
static/js/(chunkId).(hash).js
1.17MB -
public/static/js/(chunkId).(hash).js
1.1MB -
static/js/cozy.(hash).js
919.45KB -
public/static/js/cozy.(hash).js
782.19KB -
(hash).js
336.11KB -
public/(hash).js
336.11KB -
services/qualificationMigration.js
283.13KB -
services/dacc.js
262.86KB -
static/js/main.(hash).js
168.15KB -
public/static/js/public.(hash).js
141.72KB -
public/static/js/lib-react.(hash).js
43.88KB -
static/js/lib-react.(hash).js
43.88KB -
public/static/css/cozy.(hash).css
31.56KB -
static/css/cozy.(hash).css
31.56KB -
public/static/js/lib-router.(hash).js
21.86KB -
static/js/lib-router.(hash).js
21.86KB -
static/css/main.(hash).css
17.04KB -
public/static/css/public.(hash).css
6.76KB -
manifest.webapp
3.04KB -
index.html
704B -
assets/manifest.json
185B -

Total files change -2B 0%

Groups updated (1)
Status Path Size Limits
**/*.js
7.61MB (-16B 0%) -
Unchanged groups (2)
Status Path Size Limits
**/*.{png,svg,ico}
2.19MB -
**/*.css
120.14KB -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@doubleface doubleface merged commit 69fe7bc into master Apr 8, 2026
6 checks passed
@doubleface doubleface deleted the feat/newSidebar branch April 8, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants